-
Notifications
You must be signed in to change notification settings - Fork 484
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add assume-role credential provider #1036
add assume-role credential provider #1036
Conversation
Can we support the "ExternalId" parameter as part of AssumeRole Query Params ? This will limit one minio client to be associated with one externalId !! |
3d8843e
to
caec179
Compare
Sure. Added now. |
9171a84
to
c564b98
Compare
f795521
to
19cc439
Compare
19cc439
to
b7dee7f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
d0cd54b
to
3c912ce
Compare
3c912ce
to
6a9a454
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested & LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fixes #817